Skip to content

Conversation

@aalmrad
Copy link

@aalmrad aalmrad commented Nov 4, 2025

could not find the filter_chain_'s length attribute so I removed it.
Also, replaced "tf2_ros/transform_broadcaster.hpp" by "tf2_ros/transform_broadcaster.h" in the #include list

mergify bot and others added 30 commits May 12, 2025 21:21
…#1723)

Co-authored-by: Christoph Fröhlich <[email protected]>
Co-authored-by: Sai Kishor Kothakota <[email protected]>
…1585) (ros-controls#1769)

Co-authored-by: Victor Coutinho Vieira Santos <[email protected]>
Co-authored-by: Christoph Fröhlich <[email protected]>
@mergify
Copy link
Contributor

mergify bot commented Nov 4, 2025

This pull request is in conflict. Could you fix it @aalmrad?

@aalmrad aalmrad changed the title removed referene to filter_chain length removed referene to filter_chain length and tf2_ros/transform_broadcaster header Nov 4, 2025
@christophfroehlich
Copy link
Member

I think this is obsolete, right?

@aalmrad
Copy link
Author

aalmrad commented Nov 10, 2025

@christophfroehlich these issues are breaking my build although i am using the latest commits of the jazzy branch

@christophfroehlich
Copy link
Member

On which distro are you trying to build? Have you followed the build instructions, and use the correct branches of all relevant repos?

@aalmrad
Copy link
Author

aalmrad commented Nov 10, 2025

i have cloned the jazzy branch of the repository before building my workspace

@christophfroehlich
Copy link
Member

please read carefully, you also have to use the correct branches of other repos from the repos file.
And you'll need to update ros-jazzy-filters according to your commits to have the filter_length API from ros/filters#90.

@aalmrad
Copy link
Author

aalmrad commented Nov 10, 2025

@christophfroehlich upgrading the tf2_ros and filters packages solved the issue. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants